Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updates to Teradata Provider #39217

Merged
merged 13 commits into from
May 29, 2024
Merged

Updates to Teradata Provider #39217

merged 13 commits into from
May 29, 2024

Conversation

sc250072
Copy link
Contributor

Implemented enhancements to the Teradata Provider for Teradata.

This release incorporates the following features:

• Introduction of Stored Procedure Support in Teradata Hook
• Inclusion of the TeradataStoredProcedureOperator for executing stored procedures
• Integration of Azure Blob Storage to Teradata Transfer Operator
• Integration of Amazon S3 to Teradata Transfer Operator
• Provision of necessary documentation, along with unit and system tests, for the Teradata Provider modifications.

Teradata Provider System Health Dashboard: https://teradata.github.io/airflow/

Teradata Provider documentation build status: https://github.com/Teradata/airflow/actions/workflows/ci-teradata-documentation.yml


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in newsfragments.

Copy link
Member

@potiuk potiuk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice set of updates !

Implemented enhancements to the Teradata Provider for Teradata.

This release incorporates the following features:

• Introduction of Stored Procedure Support in Teradata Hook
• Inclusion of the TeradataStoredProcedureOperator for executing stored procedures
• Integration of Azure Blob Storage to Teradata Transfer Operator
• Integration of Amazon S3 to Teradata Transfer Operator
• Provision of necessary documentation, along with unit and system tests, for the Teradata Provider modifications.
@sc250072
Copy link
Contributor Author

sc250072 commented May 2, 2024

Reopening due to accidental closure.

@sc250072 sc250072 reopened this May 2, 2024
Addressed following PR review comments

Azure and Amazon optional dependency in teradata provider.yaml
Added another parameter to s3 transfer operator to specify given bucket is public or not
Added more examples of stored procedure operator in stored procedure system DAG
Changed """ to " for single line SQL statements in system tests DAGs
Applied dedent for SQL statements in cloud transfer operators
@sc250072 sc250072 requested review from eladkal and Taragolis May 3, 2024 18:34
* Modified conn_id to teradata_conn_id for Teradata Operator

* static check issue fixed
Copy link
Member

@potiuk potiuk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@potiuk
Copy link
Member

potiuk commented May 27, 2024

Can you also rebase please @satish-chinthanippu ?. We want to make sure all tests pass also with latest main

@sc250072
Copy link
Contributor Author

Can you also rebase please @satish-chinthanippu ?. We want to make sure all tests pass also with latest main

rebase done. All tests are pass with latest main.

@potiuk
Copy link
Member

potiuk commented May 29, 2024

@Taragolis @eladkal -> I reviewed and resolved all the conversations (they looked like answered/resolved) - will merge this one, byt if you have other comments possibly those can be done as a follow-up.

@potiuk potiuk merged commit 0b9232e into apache:main May 29, 2024
110 checks passed
fdemiane pushed a commit to fdemiane/airflow that referenced this pull request Jun 6, 2024
* Updates to Teradata Provider with release 2.2.0

Implemented enhancements to the Teradata Provider for Teradata.

This release incorporates the following features:

• Introduction of Stored Procedure Support in Teradata Hook
• Inclusion of the TeradataStoredProcedureOperator for executing stored procedures
• Integration of Azure Blob Storage to Teradata Transfer Operator
• Integration of Amazon S3 to Teradata Transfer Operator
• Provision of necessary documentation, along with unit and system tests, for the Teradata Provider modifications.

Azure and Amazon optional dependency in teradata provider.yaml
Added another parameter to s3 transfer operator to specify given bucket is public or not
Added more examples of stored procedure operator in stored procedure system DAG
Changed """ to " for single line SQL statements in system tests DAGs
Applied dedent for SQL statements in cloud transfer operators
jannisko pushed a commit to jannisko/airflow that referenced this pull request Jun 15, 2024
* Updates to Teradata Provider with release 2.2.0

Implemented enhancements to the Teradata Provider for Teradata.

This release incorporates the following features:

• Introduction of Stored Procedure Support in Teradata Hook
• Inclusion of the TeradataStoredProcedureOperator for executing stored procedures
• Integration of Azure Blob Storage to Teradata Transfer Operator
• Integration of Amazon S3 to Teradata Transfer Operator
• Provision of necessary documentation, along with unit and system tests, for the Teradata Provider modifications.

Azure and Amazon optional dependency in teradata provider.yaml
Added another parameter to s3 transfer operator to specify given bucket is public or not
Added more examples of stored procedure operator in stored procedure system DAG
Changed """ to " for single line SQL statements in system tests DAGs
Applied dedent for SQL statements in cloud transfer operators
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants